Consider dropping tags in this repository (v0.1.0, v0.2.0, etc.) as they are no longer used.

Done for the safe subset: v0.2.0 and v0.2.1 are gone — nothing referenced them anymore. v0.1.0 has to stay for now: the main-branch .pre-commit-config.yaml of 15 repos still pins rev: v0.1.0, so retiring it today would break pre-commit CI on main across the org. It becomes unreferenced exactly when the TRI-1100 PRs merge (they replace the reference with the org .github repo hooks) — I'll retire it then as part of the post-merge cleanup.

@greptileai

Greptile Summary

This PR migrates the add-license pre-commit hook and its backing script (tools/add_copyright.py) out of this repository into the org-wide triton-inference-server/.github repo, making this repo a plain consumer of centralized tooling. The CI layer is also modernized: the pre-commit workflow gains a cache step and uses null-delimited file filtering, and a new thin conventional-pr.yml caller workflow is added for org-wide PR title validation.

• Hook migration: .pre-commit-config.yaml now pulls add-license from triton-inference-server/.github@v1.4.3 instead of the local v0.1.0 tag; tools/add_copyright.py is deleted as a result.
• CI improvements: pre-commit.yml drops the custom modified-files shell step in favor of git diff --name-only -z --diff-filter=d HEAD^1 HEAD | xargs -0 (space-safe, deletion-safe), adds a .pre-commit-config.yaml-keyed cache, and locks down job permissions to contents: read.
• New workflow: conventional-pr.yml is a thin caller for the org reusable workflow, correctly routing same-repo PRs through the safe pull_request event and fork PRs through pull_request_target (with an explicit if guard); default_install_hook_types and the conventional-pre-commit commit-msg hook are added to enforce Conventional Commits locally.

Confidence Score: 5/5

Safe to merge — the changes are a straightforward hook migration and CI modernisation with no functional regressions.

All four changed files implement clean, well-scoped changes: the add-license hook moves to the org .github repo at a pinned immutable tag, the pre-commit workflow gains space-safe file filtering and a config-keyed cache, and the new conventional-pr.yml caller correctly guards pull_request_target to fork-only PRs. No logic that could silently break existing consumers was altered.

No files require special attention.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant GH as GitHub Actions
    participant PreCommitWF as pre-commit.yml
    participant ConvPRWF as conventional-pr.yml
    participant OrgWF as .github reusable workflow (v1.4.3)
    participant PC as pre-commit (local cache)

    Dev->>GH: Open / push PR
    GH->>PreCommitWF: trigger (pull_request)
    PreCommitWF->>PreCommitWF: "checkout (fetch-depth=2)"
    PreCommitWF->>PC: restore cache (~/.cache/pre-commit keyed on config hash)
    PreCommitWF->>PreCommitWF: "git diff HEAD^1 HEAD (null-delimited, --diff-filter=d)"
    PreCommitWF->>PC: pre-commit run --files changed files
    PC-->>PreCommitWF: pass / fail (show diff on failure)

    GH->>ConvPRWF: trigger (pull_request OR pull_request_target)
    Note over ConvPRWF: if guard: same-repo → pull_request, fork → pull_request_target
    ConvPRWF->>OrgWF: "uses conventional-pr.yml@v1.4.3 (permissions: PR write, issues write)"
    OrgWF->>GH: validate PR title (Conventional Commits)
    OrgWF->>GH: apply / remove labels via API
    OrgWF-->>ConvPRWF: result

    Dev->>Dev: git commit (local)
    Note over Dev,PC: commit-msg hook fires (conventional-pre-commit v4.4.0)
    Dev->>PC: validate commit message format
    PC-->>Dev: pass / fail

_{Reviews (2): Last reviewed commit: "ci: harden CI workflows and configs per ..." | Re-trigger Greptile}

Closing: the team is moving away from centralized org-level configuration in favor of per-repository self-contained setups (see triton-inference-server/server#8897 for the first decentralized implementation). Branch retained for reference. TRI-1100